Skip to content

fix: Issue cleanup#92

Merged
matthewpeterkort merged 5 commits into
mainfrom
fix/serve-issues
May 19, 2026
Merged

fix: Issue cleanup#92
matthewpeterkort merged 5 commits into
mainfrom
fix/serve-issues

Conversation

@lbeckman314
Copy link
Copy Markdown
Contributor

@lbeckman314 lbeckman314 commented May 19, 2026

Overview 🌀

This PR replaces the custom scripts/prepare_docs.py staging approach with mkdocs-multirepo-plugin, and resolves all Zensical build warnings across imported and legacy docs.

Current Behavior ⚠️

Upstream tool docs (git-drs, syfon) are synced into .generated/docs/ via scripts/prepare_docs.py before each build. This staging step is required locally and in CI, and Zensical reports 22 warnings from bracketed literals in legacy docs content.

python scripts/prepare_docs.py
zensical build --clean

New Behavior ✔️

mkdocs-multirepo-plugin fetches upstream tool docs from GitHub automatically at build time. Zensical reads directly from docs/ with no staging step, and builds cleanly with 0 warnings.

zensical build --clean

Build pipeline

  • Removed scripts/prepare_docs.py and the make prepare / make sync targets
  • Added mkdocs-multirepo-plugin to requirements.txt and configured it in zensical.toml under [plugins.multirepo]

Docs content fixes (22 → 0 warnings)

  • Wrapped Go type literals ([]string, map[string]string, etc.) in backticks in sifter docs
  • Escaped bracket expressions in grip jsonpath and clients docs
  • Fixed [SQLite] missing URL and duplicate [psql] definition in grip databases doc
  • Removed unused link definitions in funnel docs
  • Changed [SIGNED] to SIGNED in calypr-admin docs

@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for calypr ready!

Name Link
🔨 Latest commit 06c523b
🔍 Latest deploy log https://app.netlify.com/projects/calypr/deploys/6a0c84cb1d12a70008c18d9d
😎 Deploy Preview https://deploy-preview-92--calypr.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@matthewpeterkort matthewpeterkort merged commit b8bb636 into main May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants